feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

kuroma6666 · 2024-08-31T12:54:30Z

close #1964

このPRの変更点と理由:

主な変更点

パッシブスキルに対して、スキルをONにしている際にマナを消費しきった際に整地スキルによってブロック破壊しない設定を追加

補足情報:

デフォルト設定値はOFF設定とする（初心者プレイヤー詰み防止）
パッシブスキル設定は永続化することとしました

kuroma6666 · 2024-09-14T17:38:33Z

@rito528
ご確認よろしくお願いします。

rito528

develop ブランチの変更を取り込んでしまっているようなので、 git rebase --rebase-merges 1_18 などを実行して develop ブランチの変更をコミット履歴から消していただけると助かります🙇‍♂️🙇‍♂️

kuroma6666 · 2024-09-15T05:27:58Z

了解です。

`breakskilltargetconfig`で扱えない事柄を扱う為のサブシステム

kuroma6666 · 2024-09-15T06:49:05Z

@rito528
developブランチの変更を取り込んでしまっていた内容については削除しました。

rito528

一旦2つ変更をお願いします

src/main/scala/com/github/unchama/seichiassist/listener/PlayerBlockBreakListener.scala

src/main/scala/com/github/unchama/seichiassist/util/BreakUtil.scala

kuroma6666 · 2024-09-21T07:48:50Z

一旦2つ変更をお願いします

@rito528
一旦変更かけました

...ssist/subsystems/breakskilltriggerconfig/application/actions/BreakSkillTriggerSettings.scala

src/main/scala/com/github/unchama/seichiassist/subsystems/mana/domain/ManaManipulation.scala

src/main/scala/com/github/unchama/seichiassist/listener/PlayerBlockBreakListener.scala

kuroma6666 · 2024-09-22T13:19:35Z

@rito528
レビュー内容について対応実施しました、再確認お願いします。

rito528

テーブル構成が変わることで今と実装がかなり変わりそうなので、一度以下の内容の変更をお願いします！

src/main/scala/com/github/unchama/seichiassist/subsystems/mana/domain/ManaManipulation.scala

src/main/scala/com/github/unchama/seichiassist/listener/PlayerBlockBreakListener.scala

src/main/resources/db/migration/V1.19.0__Create_break_trigger_preference_table.sql

*GiganticMinecraft#2370 (comment)

kuroma6666 · 2024-10-12T07:57:07Z

テーブル構成が変わることで今と実装がかなり変わりそうなので、一度以下の内容の変更をお願いします！

@rito528
一旦変更をかけたので、確認お願いします。

...unchama/seichiassist/subsystems/breakskilltriggerconfig/domain/BreakSkillTriggerConfig.scala

...hama/seichiassist/subsystems/breakskilltriggerconfig/domain/BreakSkillTriggerConfigKey.scala

...bsystems/breakskilltriggerconfig/infrastructure/JdbcBreakSkillTriggerConfigPersistence.scala

src/main/scala/com/github/unchama/seichiassist/subsystems/mana/domain/ManaAmount.scala

src/main/scala/com/github/unchama/seichiassist/subsystems/mana/domain/ManaManipulation.scala

...ain/scala/com/github/unchama/seichiassist/subsystems/mana/domain/LevelCappedManaAmount.scala

…sを変更

…mの修正

kuroma6666 · 2024-12-14T13:38:02Z

@rito528
ご指摘の内容を反映しました。
再度レビューよろしくお願いします。

rito528

Otherwise LGTM.

再レビュー申請前に sbt scalafixAll; scalafmtAll; を実行してもらえると助かります！

rito528 · 2024-12-14T15:32:55Z

.../breaksuppressionpreference/application/BreakSuppressionPreferenceRepositoryDefinition.scala

+  ): RepositoryDefinition[F, Player, Ref[F, BreakSuppressionPreference]] =
+    RefDictBackedRepositoryDefinition
+      .usingUuidRefDict[F, Player, BreakSuppressionPreference](persistence)(
+        BreakSuppressionPreference(doBreakSuppression = false)


ここは Repository に保存する場合の初期値を示していますが、doBreakSuppression のデフォルト値が false であるということがドメインとして定義されているので、そこに合わせると良いと思います！

Suggested change

BreakSuppressionPreference(doBreakSuppression = false)

BreakSuppressionPreference.initial

rito528 · 2024-12-14T15:49:25Z

...ms/breaksuppressionpreference/infrastructure/JdbcBreakSuppressionPreferencePersistence.scala

+    val doBreakSuppression = DB.readOnly { implicit session =>
+      sql"SELECT do_break_suppression_due_to_mana FROM player_break_suppression_preference WHERE uuid = ${key.toString}"
+        .map(rs => rs.boolean("do_break_suppression_due_to_mana"))
+        .single()
+    }
+
+    doBreakSuppression.map(BreakSuppressionPreference)


Suggested change

val doBreakSuppression = DB.readOnly { implicit session =>

sql"SELECT do_break_suppression_due_to_mana FROM player_break_suppression_preference WHERE uuid = ${key.toString}"

.map(rs => rs.boolean("do_break_suppression_due_to_mana"))

.single()

}

doBreakSuppression.map(BreakSuppressionPreference)

DB.readOnly { implicit session =>

sql"SELECT do_break_suppression_due_to_mana FROM player_break_suppression_preference WHERE uuid = ${key.toString}"

.map(_.boolean("do_break_suppression_due_to_mana"))

.single()

.map(BreakSuppressionPreference)

}

rito528 · 2024-12-14T15:51:05Z

...ain/scala/com/github/unchama/seichiassist/subsystems/mana/domain/LevelCappedManaAmount.scala

@@ -21,7 +21,6 @@ case class LevelCappedManaAmount private (manaAmount: ManaAmount, level: SeichiL
  )(manaMultiplier: ManaMultiplier): Option[LevelCappedManaAmount] = {
    manaAmount.tryUse(amount)(manaMultiplier).map(LevelCappedManaAmount(_, level))
  }
-


Suggested change

rito528 · 2024-12-14T16:03:16Z

...ain/scala/com/github/unchama/seichiassist/subsystems/breaksuppressionpreference/System.scala

+                  breakSuppressionPreferenceRepository(player).update(pref =>
+                    pref.copy(doBreakSuppression = !pref.doBreakSuppression)
+                  )


Suggested change

breakSuppressionPreferenceRepository(player).update(pref =>

pref.copy(doBreakSuppression = !pref.doBreakSuppression)

)

breakSuppressionPreferenceRepository(player)

.update(_.toggleBreakSuppression())

rito528 · 2024-12-14T16:34:56Z

src/main/scala/com/github/unchama/seichiassist/menus/skill/PassiveSkillMenu.scala

+      originalBreakStopConfig <- breakSuppressionPreferenceAPI.isBreakSuppressionEnabled(player)
+


Suggested change

originalBreakStopConfig <- breakSuppressionPreferenceAPI.isBreakSuppressionEnabled(player)

isBreakSuppressionEnabled <- breakSuppressionPreferenceAPI.isBreakSuppressionEnabled(player)

rito528 · 2024-12-15T04:19:52Z

src/main/scala/com/github/unchama/seichiassist/menus/skill/PassiveSkillMenu.scala

+              if (!originalBreakStopConfig) {
+                SequentialEffect(
+                  MessageEffect(s"${GREEN}マナが切れたらブロック破壊を止めるスキルを有効化しました。"),
+                  FocusedSoundEffect(Sound.BLOCK_STONE_BUTTON_CLICK_ON, 1f, 1f)
+                )
+              } else {
+                SequentialEffect(
+                  MessageEffect(s"${RED}マナが切れたらブロック破壊を止めるスキルを無効化しました。"),
+                  FocusedSoundEffect(Sound.BLOCK_STONE_BUTTON_CLICK_ON, 1f, 0.5f)
+                )
+              }


以下のように書いたほうが、より読みやすくなると思います

Suggested change

if (!originalBreakStopConfig) {

SequentialEffect(

MessageEffect(s"${GREEN}マナが切れたらブロック破壊を止めるスキルを有効化しました。"),

FocusedSoundEffect(Sound.BLOCK_STONE_BUTTON_CLICK_ON, 1f, 1f)

)

} else {

SequentialEffect(

MessageEffect(s"${RED}マナが切れたらブロック破壊を止めるスキルを無効化しました。"),

FocusedSoundEffect(Sound.BLOCK_STONE_BUTTON_CLICK_ON, 1f, 0.5f)

)

}

if (originalBreakStopConfig) {

SequentialEffect(

MessageEffect(s"${RED}マナが切れたらブロック破壊を止めるスキルを無効化しました。"),

FocusedSoundEffect(Sound.BLOCK_STONE_BUTTON_CLICK_ON, 1f, 0.5f)

)

} else {

SequentialEffect(

MessageEffect(s"${GREEN}マナが切れたらブロック破壊を止めるスキルを有効化しました。"),

FocusedSoundEffect(Sound.BLOCK_STONE_BUTTON_CLICK_ON, 1f, 1f)

)

}

rito528 · 2024-12-15T04:35:43Z

src/main/scala/com/github/unchama/seichiassist/listener/PlayerBlockBreakListener.scala

+   * マナ切れブロック破壊設定が `true` になっている場合、プレイヤーに破壊抑制メッセージを送信する。
+   * @param player マナ切れブロック破壊停止設定を取得するプレイヤー
+   */
+  def isBreakBlockManaFullyConsumed(player: Player): IO[Boolean] = {


見た限り PlayerBlockBreakListener 内の定義でしか使われていないようなので private にして良いと思います！(私の見落としだったらそのままで大丈夫です！)

Suggested change

def isBreakBlockManaFullyConsumed(player: Player): IO[Boolean] = {

private def isBreakBlockManaFullyConsumed(player: Player): IO[Boolean] = {

kuroma6666 force-pushed the feat-digging-stop-mana-fully-consumed branch 2 times, most recently from 14a859f to 571abd3 Compare September 14, 2024 17:30

kuroma6666 marked this pull request as ready for review September 14, 2024 17:38

rito528 requested changes Sep 15, 2024

View reviewed changes

kuroma6666 force-pushed the feat-digging-stop-mana-fully-consumed branch 2 times, most recently from bb09299 to 509644b Compare September 15, 2024 06:27

kuroma6666 added 5 commits September 15, 2024 15:42

feat: パッシブスキルメニューにマナが無くなったら掘れなくするスキルのトグルボタンを追加

1fc70ab

feat: マナを消費しきった際の分岐処理を追加

09a17d6

chore: V1.19.0 ブロック破壊の抑制トリガー設定を保存するマイグレーションファイルを追加

924113e

feat: breakskilltriggerconfigサブシステムの追加

9f45f42

`breakskilltargetconfig`で扱えない事柄を扱う為のサブシステム

feat: ブロック破壊時、「マナ切れ」かどうか判定する処理を追加

e7c7cd3

kuroma6666 force-pushed the feat-digging-stop-mana-fully-consumed branch from 509644b to e7c7cd3 Compare September 15, 2024 06:46

rito528 requested changes Sep 15, 2024

View reviewed changes

src/main/scala/com/github/unchama/seichiassist/listener/PlayerBlockBreakListener.scala Outdated Show resolved Hide resolved

src/main/scala/com/github/unchama/seichiassist/util/BreakUtil.scala Outdated Show resolved Hide resolved

kuroma6666 added 3 commits September 16, 2024 12:56

fix: isBreakBlockManaFullyConsumedをbreakskilltriggerconfigサブシステムに実装

6725736

fix: スキル発動分のマナ不足のとき、ブロック破壊処理を終了するようにする

c043e8d

fix: 消費マナ不足処理の修正

f1e0599

kuroma6666 requested a review from rito528 September 21, 2024 07:48

rito528 requested changes Sep 21, 2024

View reviewed changes

kuroma6666 added 7 commits September 22, 2024 01:59

fix: PlayerBlockBreakListenerへマナ切れブロック破壊停止設定取得処理を移動

66d94e1

fix: マナ消費できるかどうか判断する関数を修正

acb3cf8

fix: isBreakBlockManaFullyConsumed関数をPlayerBlockBreakListenerへ定義

9753a7a

chore: importするパッケージの修正

05c80c1

fix: 消費マナ不足判定処理とブロック破壊時処理を明確にした

d79bad1

chore: scalafmt

ad5b883

chore: BreakSkillTriggerSettingsの削除

02c9368

kuroma6666 requested a review from rito528 September 22, 2024 13:19

rito528 requested changes Sep 22, 2024

View reviewed changes

kuroma6666 added 4 commits October 12, 2024 16:09

chore: canAcquireのコメント修正

ae8d4c2

fix: 「マナがないときにスキルの発動を抑制するかどうか」のテーブル構造を修正

914bf47

fix isBreakBlockManaFullyConsumedの処理を修正

eaec16e

fix: 指摘内容(*)を反映し、onPlayerActiveSkillEventの処理を修正

df86061

*GiganticMinecraft#2370 (comment)

rito528 requested changes Oct 12, 2024

View reviewed changes

kuroma6666 added 7 commits December 14, 2024 15:35

fix: BreakSkillTriggerConfigKey->BreakSuppressionPreferenceへcase clas…

5d5bcd0

…sを変更

fix: BreakSkillTriggerConfigKey定義の削除

a4451d0

fix: BreakSkillTrigger->BreakSuppressionPreferenceへの変更に伴うsubsyste…

38c29da

…mの修正

fix: BreakSkillTrigger->BreakSuppressionPreferenceへの変更に伴うビルドエラー修正

7b3c3c7

fix: DBマイグレーションSQLの修正

a10401e

fix: tryConsumeの削除

1680ffe

fix: canAcuireの修正

95d772f

kuroma6666 requested a review from rito528 December 14, 2024 13:38

rito528 requested changes Dec 15, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

kuroma6666 commented Aug 31, 2024 •

edited

Loading

kuroma6666 commented Sep 14, 2024

rito528 left a comment

kuroma6666 commented Sep 15, 2024

kuroma6666 commented Sep 15, 2024

rito528 left a comment

kuroma6666 commented Sep 21, 2024 •

edited

Loading

kuroma6666 commented Sep 22, 2024

rito528 left a comment

kuroma6666 commented Oct 12, 2024

kuroma6666 commented Dec 14, 2024

rito528 left a comment

rito528 Dec 14, 2024

rito528 Dec 14, 2024

rito528 Dec 14, 2024

rito528 Dec 14, 2024

rito528 Dec 14, 2024

rito528 Dec 15, 2024

rito528 Dec 15, 2024

	BreakSuppressionPreference(doBreakSuppression = false)
	BreakSuppressionPreference.initial

		originalBreakStopConfig <- breakSuppressionPreferenceAPI.isBreakSuppressionEnabled(player)

	originalBreakStopConfig <- breakSuppressionPreferenceAPI.isBreakSuppressionEnabled(player)
	isBreakSuppressionEnabled <- breakSuppressionPreferenceAPI.isBreakSuppressionEnabled(player)

	def isBreakBlockManaFullyConsumed(player: Player): IO[Boolean] = {
	private def isBreakBlockManaFullyConsumed(player: Player): IO[Boolean] = {

feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

Are you sure you want to change the base?

feat: マナが無くなったら掘れなくするパッシブスキルを追加 #2370

Conversation

kuroma6666 commented Aug 31, 2024 • edited Loading

このPRの変更点と理由:

主な変更点

補足情報:

kuroma6666 commented Sep 14, 2024

rito528 left a comment

Choose a reason for hiding this comment

kuroma6666 commented Sep 15, 2024

kuroma6666 commented Sep 15, 2024

rito528 left a comment

Choose a reason for hiding this comment

kuroma6666 commented Sep 21, 2024 • edited Loading

kuroma6666 commented Sep 22, 2024

rito528 left a comment

Choose a reason for hiding this comment

kuroma6666 commented Oct 12, 2024

kuroma6666 commented Dec 14, 2024

rito528 left a comment

Choose a reason for hiding this comment

rito528 Dec 14, 2024

Choose a reason for hiding this comment

rito528 Dec 14, 2024

Choose a reason for hiding this comment

rito528 Dec 14, 2024

Choose a reason for hiding this comment

rito528 Dec 14, 2024

Choose a reason for hiding this comment

rito528 Dec 14, 2024

Choose a reason for hiding this comment

rito528 Dec 15, 2024

Choose a reason for hiding this comment

rito528 Dec 15, 2024

Choose a reason for hiding this comment

kuroma6666 commented Aug 31, 2024 •

edited

Loading

kuroma6666 commented Sep 21, 2024 •

edited

Loading